Skip to content

PYTHON-4928 Convert CSFLE spec tests to unified test format#2520

Merged
blink1073 merged 14 commits intomongodb:masterfrom
blink1073:PYTHON-4928-2
Sep 8, 2025
Merged

PYTHON-4928 Convert CSFLE spec tests to unified test format#2520
blink1073 merged 14 commits intomongodb:masterfrom
blink1073:PYTHON-4928-2

Conversation

@blink1073
Copy link
Member

No description provided.

@blink1073 blink1073 requested a review from NoahStapp September 8, 2025 17:05
@blink1073 blink1073 requested a review from a team as a code owner September 8, 2025 17:05
async def _collectionOperation_assertCollectionExists(self, target, **kwargs):
database_name = kwargs["database_name"]
collection_name = kwargs["collection_name"]
collection_name_list = list(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

list_collection_names already returns a list.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

async def _databaseOperation_assertCollectionExists(self, target, **kwargs):
database_name = kwargs["database_name"]
collection_name = kwargs["collection_name"]
collection_name_list = list(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

list_collection_names already returns a list.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@blink1073 blink1073 requested a review from NoahStapp September 8, 2025 17:28
NoahStapp
NoahStapp previously approved these changes Sep 8, 2025
@blink1073
Copy link
Member Author

Had to update another case sensitivity issue.

@blink1073 blink1073 merged commit 7580309 into mongodb:master Sep 8, 2025
78 of 80 checks passed
@blink1073 blink1073 deleted the PYTHON-4928-2 branch September 8, 2025 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments